Skip to content

Faster image build#124

Merged
rafaelfranca merged 7 commits intomainfrom
rm-faster-buiolds
Apr 23, 2026
Merged

Faster image build#124
rafaelfranca merged 7 commits intomainfrom
rm-faster-buiolds

Conversation

@rafaelfranca
Copy link
Copy Markdown
Member

Make image build faster by avoid using emulation to build arm64 images. Instead, have the image built in arm64 ubuntu machines.

The actions also got improved removing duplication and making easier to trigger in the interface.

…c images in parallel

Then creating and pushing a multi-arch manifest in a separate step.
This allows us to build for amd64 and arm64 simultaneously,
reducing overall build time.
Comment thread .github/workflows/publish-new-ruby-versions.yaml Outdated
Comment thread .github/workflows/publish-new-ruby-versions.yaml Outdated
Copy link
Copy Markdown
Member

@andrewn617 andrewn617 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is great.

Just a couple suggestions but glad we are improving this :D

@rafaelfranca
Copy link
Copy Markdown
Member Author

@copilot can you address the comments here?

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 22, 2026

@copilot can you address the comments here?

Addressed in c6f5c2c. I updated the workflow to set up Ruby from .ruby-version and moved interpolated inputs into env variables for safer command usage. No UI changes to screenshot.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • apt.postgresql.org
    • Triggering command: /usr/bin/curl curl -s REDACTED grep -F um.pem rvices_root.crt /usr/bin/ln .conf mktemp table.d/chrony-o ln -sf es_root.pem ot_R46.crt /usr/bin/ln 4.deb p.ci plit ln (dns block)
    • Triggering command: /usr/lib/apt/methods/https /usr/lib/apt/methods/https /var/lib/dpkg/tm-x (dns block)
  • containers.dev
    • Triggering command: /home/REDACTED/work/_temp/ghcca-node/node/bin/node node /home/REDACTED/work/devcontainer/devcontainer/node_modules/.bin/devcontainer features test features -i ubuntu:noble (dns block)
  • docker-images-prod.6aa30f8b08e16409b46e0173d6de2f56.r2.cloudflarestorage.com
    • Triggering command: /home/REDACTED/work/_temp/ghcca-node/node/bin/node node /home/REDACTED/work/devcontainer/devcontainer/node_modules/.bin/devcontainer features test features -i ubuntu:noble (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

@rafaelfranca
Copy link
Copy Markdown
Member Author

@copilot the environment variables are still being passed as arguments. We should just read the environment in the bin/normalize-publish-inputs file. Can you change that? Only read from the env if the arguments aren't passed, but, in the github action, don't pass any argument

Copilot AI and others added 2 commits April 23, 2026 00:47
Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 23, 2026

@copilot the environment variables are still being passed as arguments. We should just read the environment in the bin/normalize-publish-inputs file. Can you change that? Only read fro...

Implemented in 764953e: the workflow now runs ruby bin/normalize-publish-inputs with no arguments, and the script reads RUBY_VERSIONS, IMAGE_VERSIONS, and REPOSITORY from env only when corresponding CLI args are not provided. I also added coverage for env fallback and arg precedence in 143747a. No UI changes to screenshot.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • apt.postgresql.org
    • Triggering command: /usr/bin/curl curl -s REDACTED grep -F um.pem rvices_root.crt /usr/bin/ln n.conf stall-C6OeOx/210configure /usr/bin/dpkg-sp ln -sf es_root.pem ot_R46.crt /usr/bin/ln 4.deb p.ci b ln (dns block)
    • Triggering command: /usr/lib/apt/methods/https /usr/lib/apt/methods/https /var/lib/dpkg/tm-x (dns block)
  • containers.dev
    • Triggering command: /home/REDACTED/work/_temp/ghcca-node/node/bin/node node /home/REDACTED/work/devcontainer/devcontainer/node_modules/.bin/devcontainer features test features -i ubuntu:noble --gdwarf-5 --64 -o regexp.o /tmp/cclK60Sn.s conf�� by-3.2.0 --global 1.9.0/include credential.helpeas 1.9.0/ext (dns block)
  • docker-images-prod.6aa30f8b08e16409b46e0173d6de2f56.r2.cloudflarestorage.com
    • Triggering command: /home/REDACTED/work/_temp/ghcca-node/node/bin/node node /home/REDACTED/work/devcontainer/devcontainer/node_modules/.bin/devcontainer features test features -i ubuntu:noble --gdwarf-5 --64 -o regexp.o /tmp/cclK60Sn.s conf�� by-3.2.0 --global 1.9.0/include credential.helpeas 1.9.0/ext (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

@rafaelfranca rafaelfranca enabled auto-merge April 23, 2026 03:10
@rafaelfranca rafaelfranca merged commit 5be7b92 into main Apr 23, 2026
36 checks passed
@rafaelfranca rafaelfranca deleted the rm-faster-buiolds branch April 23, 2026 03:15
@rafaelfranca
Copy link
Copy Markdown
Member Author

@copilot hmmm. It seems publish-images-reusable is appearing as a worflow you can run. It should not

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants